Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] 디자인 시스템 기반 컴포넌트 일부 구현 #8

Merged
merged 4 commits into from
May 18, 2024

Conversation

cobocho
Copy link
Contributor

@cobocho cobocho commented May 18, 2024

PR Type

  • 기능 추가

Overview

  • UI 패키지명을 @vook-client/ui 에서 @vook-client/design-system으로 변경
  • 디자인 토큰 추출 및 sprinkles 생성
  • Text 컴포넌트 구현
  • Button 컴포넌트 구현

Additional Comment

Issue

#7

Check List

  • 관련된 이슈를 연결하였나요?
  • 올바른 PR 컨벤션을 작성했나요?
  • 적절한 라벨을 선택했나요?
  • Assignee와 Reviewer를 설정했나요?

References

@cobocho cobocho added the 🧑‍💻 Feature 기능 개발 및 삭제 관련 사항입니다 label May 18, 2024
@cobocho cobocho requested a review from hin6150 May 18, 2024 10:51
Copy link
Member

@hin6150 hin6150 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

테스트 공부하시더니, 테스트 코드까지 작성해두셨군요.. 👨‍💻
열심히 한번 따라가보겠습니다! :)

@hin6150 hin6150 merged commit cea3dd8 into main May 18, 2024
3 checks passed
@cobocho cobocho deleted the design-system branch May 18, 2024 12:13
Comment on lines +111 to +116
<li>
<p className="storybook-subtitle">No Blue Line</p>
<Button {...props} filled={false}>
{props.children}
</Button>
</li>
Copy link
Member

@hin6150 hin6150 May 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

엄청 중요한 부분은 아닌데 작업하면서 보니
<Button {...props} filled={false} blueLine={false}>
으로 작성해야 Story Book에서 blueline이 제거된 상태로 올바르게 표기되더라구요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 Feature 기능 개발 및 삭제 관련 사항입니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants